-
Notifications
You must be signed in to change notification settings - Fork 39
test: add scalingo-24
#528
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
68694a1
to
cd93ca8
Compare
94ed08f
to
75e6d82
Compare
75e6d82
to
5f885b8
Compare
# | ||
|
||
NGINX_VERSION="1.22.1" | ||
NGINX_VERSION="1.28.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated these as per @Frzk suggestion
@@ -1,4 +1,4 @@ | |||
#!/bin/sh | |||
#!/usr/bin/env bash |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bash is needed for support of :
in function name
506f372
to
e7c59e5
Compare
strategy: | ||
matrix: | ||
stack: | ||
- scalingo-20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Frzk confirmed that we can remove scalingo-20 (https://scalingo.slack.com/archives/C08LD7JN22V/p1756808049686519?thread_ts=1756805070.647969&cid=C08LD7JN22V)
No description provided.